xen-netback: Disable NAPI after disabling interrupts
authorZoltan Kiss <zoltan.kiss@linaro.org>
Tue, 28 Oct 2014 15:29:30 +0000 (15:29 +0000)
committermaximilian attems <maks@debian.org>
Sat, 31 Jan 2015 21:20:40 +0000 (21:20 +0000)
Otherwise the interrupt handler still calls napi_complete. Although it
won't schedule NAPI again as either NAPI_STATE_DISABLE or
NAPI_STATE_SCHED is set, it is just unnecessary, and it makes more
sense to do this way.

Signed-off-by: Zoltan Kiss <zoltan.kiss@linaro.org>
Signed-off-by: David Vrabel <david.vrabel@citrix.com>
Acked-by: Wei Liu <wei.liu2@citrix.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Gbp-Pq: Topic bugfix/all
Gbp-Pq: Name xen-netback-Disable-NAPI-after-disabling-interrupts.patch

drivers/net/xen-netback/interface.c

index d752d1c5c8bd9602e68b9824c308449c59747baa..9259a732e8a4a6d20740a1ad9170962040b8d1f8 100644 (file)
@@ -235,10 +235,10 @@ static void xenvif_down(struct xenvif *vif)
 
        for (queue_index = 0; queue_index < num_queues; ++queue_index) {
                queue = &vif->queues[queue_index];
-               napi_disable(&queue->napi);
                disable_irq(queue->tx_irq);
                if (queue->tx_irq != queue->rx_irq)
                        disable_irq(queue->rx_irq);
+               napi_disable(&queue->napi);
                del_timer_sync(&queue->credit_timeout);
        }
 }